Skip to content

nextpnr-xilinx: init at 0.8.2-unstable-2026-02-25 - #494173

Merged
pbsds merged 1 commit into
NixOS:masterfrom
RCoeurjoly:nextpnr-xilinx-nixpkgs
Mar 7, 2026
Merged

nextpnr-xilinx: init at 0.8.2-unstable-2026-02-25 #494173
pbsds merged 1 commit into
NixOS:masterfrom
RCoeurjoly:nextpnr-xilinx-nixpkgs

Conversation

@RCoeurjoly

@RCoeurjoly RCoeurjoly commented Feb 25, 2026 •
edited by pbsds
Loading

Copy link
Copy Markdown
Contributor

Apart from the postPatch, all comes from https://github.com/openXC7/toolchain-nix/blob/main/nix/nextpnr-xilinx.nix.

Previous effort was here, but now stale: #257737

pbsds edit: closes #257737

Things done

  • Built on platform:
    • x86_64-linux
    • aarch64-linux
    • x86_64-darwin
    • aarch64-darwin
  • Tested, as applicable:
  • Ran nixpkgs-review on this PR. See nixpkgs-review usage.
  • Tested basic functionality of all binary files, usually in ./result/bin/.
  • Nixpkgs Release Notes
    • Package update: when the change is major or breaking.
  • NixOS Release Notes
    • Module addition: when adding a new NixOS module.
    • Module update: when the change is significant.
  • Fits CONTRIBUTING.md, pkgs/README.md, maintainers/README.md and other READMEs.

@RCoeurjoly

RCoeurjoly commented Feb 25, 2026

Copy link
Copy Markdown
Contributor Author

I added you @jleightcap and @albertchae because you were involved in this some time ago. Thanks for that!

@nixpkgs-ci nixpkgs-ci Bot added 8.has: package (new) This PR adds a new package 10.rebuild-linux: 1-10 This PR causes between 1 and 10 packages to rebuild on Linux. 10.rebuild-darwin: 1-10 This PR causes between 1 and 10 packages to rebuild on Darwin. 10.rebuild-darwin: 1 This PR causes 1 package to rebuild on Darwin. 10.rebuild-linux: 1 This PR causes 1 package to rebuild on Linux. labels Feb 25, 2026
qweered
qweered reviewed Feb 25, 2026
View reviewed changes
Comment thread pkgs/by-name/ne/nextpnr-xilinx/package.nix Outdated
qweered
qweered reviewed Feb 25, 2026
View reviewed changes
Comment thread pkgs/by-name/ne/nextpnr-xilinx/package.nix Outdated
qweered
qweered reviewed Feb 25, 2026
View reviewed changes
Comment thread pkgs/by-name/ne/nextpnr-xilinx/package.nix
qweered
qweered suggested changes Feb 25, 2026
View reviewed changes

@qweered qweered left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Also needs nix fmt run

qweered
qweered reviewed Feb 25, 2026
View reviewed changes
Comment thread pkgs/by-name/ne/nextpnr-xilinx/package.nix Outdated
qweered
qweered reviewed Feb 25, 2026
View reviewed changes
Comment thread pkgs/by-name/ne/nextpnr-xilinx/package.nix
@RCoeurjoly
RCoeurjoly force-pushed the nextpnr-xilinx-nixpkgs branch from 16bf5fd to 6cf1222 Compare February 25, 2026 23:12
@RCoeurjoly

RCoeurjoly commented Feb 25, 2026

Copy link
Copy Markdown
Contributor Author

Also needs nix fmt run

Done, thanks!

@RCoeurjoly
RCoeurjoly requested a review from qweered February 25, 2026 23:14
@qweered

qweered commented Feb 25, 2026

Copy link
Copy Markdown
Contributor

nixpkgs-review result

Generated using nixpkgs-review-gha

Command: nixpkgs-review pr 494173
Commit: 6cf12224f1c31cf29b5cea589912c8843992d2de (subsequent changes)
Merge: 639e9d51332b528beb9aedd758d32062a931b2ed

Logs: https://github.com/qweered/nixpkgs-review-gha/actions/runs/22420858292

x86_64-linux

✅ 1 package built:
  • nextpnr-xilinx

aarch64-linux

✅ 1 package built:
  • nextpnr-xilinx

aarch64-darwin (sandbox = false)

✅ 1 package built:
  • nextpnr-xilinx

qweered
qweered approved these changes Feb 25, 2026
View reviewed changes

@qweered qweered left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Approved automatically following the successful run of nixpkgs-review.

@nixpkgs-ci nixpkgs-ci Bot added the 12.approvals: 1 This PR was reviewed and approved by one person. label Feb 25, 2026
@RCoeurjoly
RCoeurjoly requested review from pbsds and removed request for albertchae and jleightcap February 28, 2026 20:00
pbsds
pbsds reviewed Mar 1, 2026
View reviewed changes
Comment thread pkgs/by-name/ne/nextpnr-xilinx/package.nix Outdated
Comment thread pkgs/by-name/ne/nextpnr-xilinx/package.nix
Comment thread pkgs/by-name/ne/nextpnr-xilinx/package.nix Outdated
@RCoeurjoly
RCoeurjoly force-pushed the nextpnr-xilinx-nixpkgs branch from 3e5550e to 982fdfe Compare March 2, 2026 20:37
@RCoeurjoly
RCoeurjoly requested a review from pbsds March 3, 2026 15:26
@pbsds pbsds changed the title nextpnr-xilinx: init at unstable-2026-02-25 nextpnr-xilinx: init at 0.8.2-unstable-2026-02-25 Mar 7, 2026
@pbsbot

pbsbot commented Mar 7, 2026

Copy link
Copy Markdown

nixpkgs-review result

Generated using nixpkgs-review.

Command: nixpkgs-review pr 494173
Commit: 982fdfe48eb5053dbe2a503c66f5f4aab47e263b

x86_64-linux

✅ 1 package built:
  • nextpnr-xilinx

@pbsds
pbsds added this pull request to the merge queue Mar 7, 2026
Merged via the queue into NixOS:master with commit 02546df Mar 7, 2026
30 of 32 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@pbsds pbsds Awaiting requested review from pbsds
1 more reviewer
@qweered qweered qweered approved these changes
Reviewers whose approvals may not affect merge requirements

Assignees

No one assigned

Labels

8.has: package (new) This PR adds a new package 10.rebuild-darwin: 1-10 This PR causes between 1 and 10 packages to rebuild on Darwin. 10.rebuild-darwin: 1 This PR causes 1 package to rebuild on Darwin. 10.rebuild-linux: 1-10 This PR causes between 1 and 10 packages to rebuild on Linux. 10.rebuild-linux: 1 This PR causes 1 package to rebuild on Linux. 12.approvals: 1 This PR was reviewed and approved by one person.

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

4 participants

@RCoeurjoly @qweered @pbsbot @pbsds